Skip to content

docs: fix navigation.json path in adding-pages guide and typos in blog posts - #9158

Open
toyeshhm wants to merge 1 commit into
nodejs:mainfrom
toyeshhm:fix-doc-typos
Open

docs: fix navigation.json path in adding-pages guide and typos in blog posts#9158
toyeshhm wants to merge 1 commit into
nodejs:mainfrom
toyeshhm:fix-doc-typos

Conversation

@toyeshhm

Copy link
Copy Markdown

Description

docs/adding-pages.md tells contributors twice (lines 76 and 95) to register new pages in app/site/navigation.json. That path does not exist; the file is apps/site/navigation.json. Anyone following the "Adding Learn Pages" guide is sent to a nonexistent file.

Also fixes misspellings in the English blog posts:

  • blog/announcements/making-nodejs-downloads-reliable.md: occuring → occurring, "being effected" → "being affected", additon → addition
  • blog/community/2025-pride.md: millenia → millennia
  • blog/events/collab-summit-2025-paris.md: rewritting → rewriting
  • blog/vulnerability/february-2024-security-releases.md: overwitten → overwritten, encounted → encountered
  • blog/vulnerability/august-2023-security-releases.md: retrive → retrieve (heading)
  • blog/vulnerability/june-2023-security-releases.md: interuption → interruption (heading)
  • blog/vulnerability/february-2023-security-releases.md: priviledges → privileges
  • blog/vulnerability/july-2022-security-releases.md: serverity → severity
  • blog/vulnerability/openssl-fixes-in-regular-releases-may2022.md: serverity → severity, delievered → delivered
  • blog/vulnerability/jan-2022-security-releases.md: protoype → prototype
  • blog/vulnerability/aug-2021-security-releases2.md: annoucement → announcement

Two of the fixes are in headings of security release posts, so their generated anchor ids change. Happy to drop those two if any inbound links depend on the old anchors.

Validation

Markdown-only changes; no code touched. Reviewed each line in context. Did not run the pnpm format/test/build steps locally since only .md files are edited; CI will confirm.

Related Issues

None.

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

…g posts

docs/adding-pages.md tells contributors twice to edit
`app/site/navigation.json`, which does not exist. The file is
`apps/site/navigation.json`.

Also fix misspellings in the English blog posts: occuring, effected,
additon, millenia, rewritting, overwitten, encounted, retrive,
interuption, priviledges, serverity, delievered, protoype, annoucement.

Signed-off-by: toyeshhm <toyeshhm@gmail.com>
@toyeshhm
toyeshhm requested review from a team as code owners September 13, 2026 03:39
@vercel

vercel Bot commented Sep 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
nodejs-org Ready Ready Preview Sep 13, 2026 3:40am UTC

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant